Skip to content

Show smoke test in test reports#93

Merged
PiotrKorkus merged 8 commits intoeclipse-score:mainfrom
etas-contrib:fix/show_smoketest_in_test_report
Mar 2, 2026
Merged

Show smoke test in test reports#93
PiotrKorkus merged 8 commits intoeclipse-score:mainfrom
etas-contrib:fix/show_smoketest_in_test_report

Conversation

@eelcoem
Copy link
Contributor

@eelcoem eelcoem commented Feb 25, 2026 •
edited
Loading

Partially fixes #92 so that the test shows up as failed. Another PR to the cicd repo is needed to make it actually pass

@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 07:25 — with GitHub Actions Inactive
@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 07:25 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Feb 25, 2026 •
edited
Loading

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.4.2) and connecting to it...
INFO: Invocation ID: 66d71da1-e587-4b31-91da-260505b76288
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'score_rust_policies', the root module requires module version score_rust_policies@0.0.3, but got score_rust_policies@0.0.5 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (49 packages loaded, 9 targets configured)

Analyzing: target //:license-check (86 packages loaded, 9 targets configured)

Analyzing: target //:license-check (140 packages loaded, 2614 targets configured)

Analyzing: target //:license-check (149 packages loaded, 5508 targets configured)

Analyzing: target //:license-check (155 packages loaded, 7768 targets configured)

Analyzing: target //:license-check (160 packages loaded, 7817 targets configured)

Analyzing: target //:license-check (160 packages loaded, 7817 targets configured)

Analyzing: target //:license-check (161 packages loaded, 7941 targets configured)

Analyzing: target //:license-check (164 packages loaded, 9829 targets configured)

INFO: Analyzed target //:license-check (165 packages loaded, 9955 targets configured).
[14 / 16] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
[15 / 16] Building license.check.license_check.jar (); 0s disk-cache, multiplex-worker
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 25.299s, Critical Path: 2.65s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 07:27 — with GitHub Actions Inactive
@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 07:27 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Feb 25, 2026

The created documentation from the pull request is available at: docu-html

@PiotrKorkus
Copy link
Contributor

PiotrKorkus commented Feb 25, 2026

Why failed test doesn't fail a workflow?
https://github.com/eclipse-score/lifecycle/actions/runs/22386709444/job/64798815601?pr=93#step:4:491

@eelcoem
Copy link
Contributor Author

eelcoem commented Feb 25, 2026

Why failed test doesn't fail a workflow? https://github.com/eclipse-score/lifecycle/actions/runs/22386709444/job/64798815601?pr=93#step:4:491

tests.yml in the cicd repo has continue-on-error: true, will fix that in the PR there

Empting Eelco (ETAS-ECM/XPC-Fe2) added 2 commits February 25, 2026 09:57
@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 09:33 — with GitHub Actions Inactive
@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 09:33 — with GitHub Actions Inactive
Empting Eelco (ETAS-ECM/XPC-Fe2) added 2 commits February 25, 2026 11:24
Revert "Fix pipeline run"
df555ec 
This reverts commit 18cf5d1.
Revert "Modify workflow"
de607dd 
This reverts commit 0552bc0.
@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 10:30 — with GitHub Actions Inactive
@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 10:30 — with GitHub Actions Inactive
@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 10:53 — with GitHub Actions Inactive
@eelcoem eelcoem temporarily deployed to workflow-approval February 25, 2026 10:53 — with GitHub Actions Inactive
@PiotrKorkus
Copy link
Contributor

PiotrKorkus commented Mar 2, 2026

retriggered tests to check if works with updated workflow after merge eclipse-score/cicd-workflows#68

PiotrKorkus
PiotrKorkus approved these changes Mar 2, 2026
View reviewed changes
Copy link
Contributor

@PiotrKorkus PiotrKorkus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

workflow failed because it is triggered on pull_request_target which takes workflow file from main branch and the called workflow from cicd-workflows repo has already been merged.

LGTM

@PiotrKorkus PiotrKorkus merged commit ee8b59a into eclipse-score:main Mar 2, 2026
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PiotrKorkus PiotrKorkus PiotrKorkus approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Failed smoke test in statistics dashboard

2 participants

@eelcoem @PiotrKorkus